]> dgit.raspbian.org Git - gnome-shell.git/commit
breakManager: Switch from lightbox to a Clutter effect for fading out
authorPhilip Withnall <pwithnall@gnome.org>
Mon, 3 Mar 2025 18:31:28 +0000 (18:31 +0000)
committerMarge Bot <marge-bot@gnome.org>
Wed, 5 Mar 2025 12:06:07 +0000 (12:06 +0000)
commit5d1157cdfaaa2b3196a4033d2ba411a0e0fb1695
tree9f8e70a91c93a7f67d39150363216fb031ec5a14
parent0d238a82e0e016a3b57ad985c8a1d86fc1c889cb
breakManager: Switch from lightbox to a Clutter effect for fading out

From some testing on a full session with break reminders enabled, it
seems that it’s possible to break the stacking order of the lightbox
actor if the user continues to interact with the session while it’s
visible (for example, while they’re supposed to be taking a movement
break).

Rather than try and work out what’s going on with the lightbox here, it
seems more robust to switch to using a Clutter effect, like we did with
the screen time UI (in `timeLimitsManager`). Then the effect can be
applied to the main UI group, and doesn’t depend on a separate actor
being placed in the right place in the scene graph.

So, remove the lightbox, and instead add a
`ClutterBrightnessSaturationEffect`, and use its brightness property to
fade the screen slightly while in breaks.

Signed-off-by: Philip Withnall <pwithnall@gnome.org>
Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3655>
js/misc/breakManager.js